Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Admin #104

Merged
merged 19 commits into from
Mar 17, 2022
Merged

Admin #104

merged 19 commits into from
Mar 17, 2022

Conversation

dmdgpdi
Copy link
Collaborator

@dmdgpdi dmdgpdi commented Mar 17, 2022

PR Category

PR 종류를 선택해주세요.

  • New Feature
  • Bug Fix
  • Refactor Code
  • Etc

Linked Issues

#이슈번호 형태로 추가해주세요.

PR Summary

변경 사항을 요약해주세요.

(why -> what -> how)

  • 기존의 권한 부여 및 제거하는 기능이 사이드바로 나뉘어서 동작하고 있었습니다.
    이를 정리하고 모달창으로 구현해 이용자가 보기에 더 쾌적하게 변경되었습니다.

Comment

추가로 논의할 내용이 있다면 적어주세요.

  • 회원 가입 승인 및 거절, 전체 동아리원 권한 초기화, 게시판 추가 및 삭제의 모달창은 UI를 먼저 구성한 뒤 그에 맞춰서 변경할
    예정입니다.

References

참고한 사이트가 있다면 공유해주세요.

Check List

  • 추가한 기능에 대한 테스트는 모두 완료했나요?
  • 코드 정렬, 불필요한 코드나 오타는 없는지 확인했나요?

@dmdgpdi dmdgpdi merged commit f9ba7c5 into main Mar 17, 2022
@dmdgpdi dmdgpdi deleted the admin branch March 17, 2022 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant